Skip to content

fix(ui): collapse Pi skill invocations - #4056

Merged
trunk-io[bot] merged 2 commits into
mainfrom
fix/pi-skill-invocation-display
Jul 31, 2026
Merged

fix(ui): collapse Pi skill invocations#4056
trunk-io[bot] merged 2 commits into
mainfrom
fix/pi-skill-invocation-display

Conversation

@jonathanlab

Copy link
Copy Markdown
Contributor

Summary

  • render Pi-expanded skill messages as compact command chips
  • retain a skill invocation’s user request after the chip
  • cover skill-block collapsing and user-message rendering

Testing

  • pnpm exec biome lint packages/ui/src/features/sessions/components/session-update/parseFileMentions.tsx packages/ui/src/features/sessions/components/session-update/parseFileMentions.test.ts packages/ui/src/features/sessions/components/session-update/UserMessage.tsx packages/ui/src/features/sessions/components/session-update/UserMessage.test.tsx packages/ui/src/features/sessions/components/chat-thread/ChatThread.tsx
  • pnpm --filter @posthog/ui test --run src/features/sessions/components/session-update/parseFileMentions.test.ts src/features/sessions/components/session-update/UserMessage.test.tsx

@trunk-io

trunk-io Bot commented Jul 31, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit d71d5e4.

@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(ui): collapse Pi skill invocations" | Re-trigger Greptile

@jonathanlab jonathanlab added the Stamphog This will request an autostamp by stamphog on small changes label Jul 31, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small, contained UI rendering fix (collapses a skill-invocation prefix in chat message display) with unit tests covering the new logic; no risky territory touched.

  • Author wrote 0% of the modified lines and has 6 merged PRs in these paths (familiarity MODERATE).
  • 👍 on the PR from greptile-apps[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 29L, 3F substantive, 58L/5F incl. docs/generated/snapshots — within ceiling
tier T1-agent / T1b-small (58L, 5F, single-area, fix)
stamphog 2.0.0b3 .stamphog/policy.yml @ 436326e · reviewed head d71d5e4

@trunk-io
trunk-io Bot merged commit 6b65b0f into main Jul 31, 2026
38 of 39 checks passed
@trunk-io
trunk-io Bot deleted the fix/pi-skill-invocation-display branch July 31, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant